-
-
Notifications
You must be signed in to change notification settings - Fork 72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Retire nightly jdk19 builds and replace with jdk20 #628
Retire nightly jdk19 builds and replace with jdk20 #628
Conversation
Also includes a few formatting changes for tidiness. Signed-off-by: Adam Farley <[email protected]>
Thank you for creating a pull request!Please check out the information below if you have not made a pull request here before (or if you need a reminder how things work). Code Quality and Contributing GuidelinesIf you have not done so already, please familiarise yourself with our Contributing Guidelines and Code Of Conduct, even if you have contributed before. TestsGithub actions will run a set of jobs against your PR that will lint and unit test your changes. Keep an eye out for the results from these on the latest commit you submitted. For more information, please see our testing documentation. In order to run the advanced pipeline tests (executing a set of mock pipelines), it requires an admin to post |
@steelhead31 - Heya Scott. I think it's a good idea to add a requirement for this sort of PR into the release checklist as a post-release event, to cover releases that are a quarterly CPU containing a short-term release with no subsequent CPUs (e.g. the January release in regard to jdk19). What do you think? |
Definitely a good idea, I've got a number of changes to make to the release checklist, which Im planning to do tomorrow, alongside a number of other documentation changes following the last CPU, so I'll be sure to include this! |
Also includes a few formatting changes for tidiness. Signed-off-by: Adam Farley <[email protected]>
Also includes a few formatting changes for tidiness.